[FEAT] CI & Swiftlint autocorrect 깃허브 액션 추가 by 0Hooni · Pull Request #98 · PopPool/iOS

0Hooni · 2025-04-02T15:08:39Z

📌 이슈

Swiftlint & CI 적용하기 #93

✅ 작업 사항

🚀 테스트 방식

Swiftlint 테스트

노션에 프로젝트 세팅 문서의 "swiftlint 적용 과정"의 1번을 참고하여 swiftlint 설치
93번 브랜치에서 프로젝트 빌드

깃허브 액션 테스트

현재 PR을 오픈하여 적용된 깃허브 액션 적용 테스트 예정

변경된 Key 관리 테스트

Swiftlint로 인해서 일반적인 상황으로는 빌드가 안됨
에러가 되는 린트 룰을 .swiftlint.yml 파일에서 비활성화 하여 빌드하여 테스트

👀 ETC (추후 개발해야 할 것, 참고자료 등) ->

Caution

이번 PR을 통해 Secrets.swift의 추적 제한이 풀렸습니다❗
PR이 병합되고 난 후 반드시 Secrets.swift를 디렉토리에서 제거하고, 작업자의 커밋에 포함되지 않았는지 꼭 확인해주세요❗❗❗

- 컨벤션 파일 제거로 인한 빌드 파일 수정

- Run SwiftLint 스크립트 추가 - SwiftLint rule 파일 추가

- Build 테스트 포함 - Swiftlint 테스트 포함

- Debug.xcconfig를 Resource 폴더에 추가함 - 해당 파일은 ignore로 걸러지도록 함 - info에 Key의 이름을 등록해두어 Bundle로 찾도록 설정

- 새로운 Debug.xcconfig를 무시하도록 수정 - 기존 ignore 설정을 보기 쉽게 정리

- Secrets.swift의 enum을 열도록 할 예정 - 해당 파일에서 Key 호출을 위한 여러 컴퓨티드 프로퍼티 제공 - 변경된 프로퍼티에 맞게 호출 코드도 일부 수정

- 깃헙 액션에 맞게 파일 이름 수정 - CI → ci - Swiftlint autocorrect → swiftlint-autocorrect - 추가로 autocorrect의 작동 시기를 Push → PR open으로 수정하여 깃액션 작동 빈도를 줄임

- Secrets enum을 대체할 KeyPath enum을 추가 - gitignore에서 Secrets.swift 추적 제한 제거 - 빌드 파일 수정

- 몇몇 이모지가 생각보다 촌스러워서 수정함

0Hooni · 2025-04-02T15:23:35Z

성공적인 첫 CI 🥹

0Hooni · 2025-04-02T16:15:17Z

🥹 자동 수정이랑 빌드 테스트 전부 작동 테스트 완료 🥹

자동 수정으로 줄어들은 warning 수(4000 → 100 실화?!?!?)

…pply-CI-with-swiftlint # Conflicts: # Poppool/Poppool/Application/AppDelegate.swift # Poppool/Poppool/Presentation/Admin/AdminRegister/PopUpStoreRegisterViewController.swift # Poppool/Poppool/Presentation/Admin/Data/MapDomain/MapPopUpStore.swift # Poppool/Poppool/Presentation/Map/Common/GMSMapViewDelegateProxy.swift # Poppool/Poppool/Presentation/Map/Common/MapUtilities.swift # Poppool/Poppool/Presentation/Map/FillterSheetView/FilterBottomSheetView.swift # Poppool/Poppool/Presentation/Map/FillterSheetView/FilterBottomSheetViewController.swift # Poppool/Poppool/Presentation/Map/FindMap/MapGuideView/FullScreenMapViewController.swift # Poppool/Poppool/Presentation/Map/FindMap/MapGuideView/MapGuideViewController.swift # Poppool/Poppool/Presentation/Map/MapView/MapMarker.swift # Poppool/Poppool/Presentation/Map/MapView/MapView.swift # Poppool/Poppool/Presentation/Map/MapView/MapViewController.swift Co-Authored-By: ki hyeon kim <creeps1104@gmail.com>

Co-Authored-By: JunYoung <ghddns34@gmail.com>

dongglehada

시크릿에서 키패스로 변경이 되었군요 변경상항은 확인 하였습니다! 정리해주신 내용 확인해보겠습니다!

dongglehada · 2025-04-04T10:59:27Z

.github/workflows/ci.yml

+            echo "No changes to commit"
+          fi
+
+  build:


Build Flow를 여기서 정의하는 걸까요..? 신기합니다!

@dongglehada

ci 내부에 여러 jobs을 둘 수 있고 현재 총 두개의 job을 두고 있습니다.

autocorrect: 해당 job에 포함된 step에 따라 swiftlint 자동 수정 및 수정 내용 커밋

build: 스텝에 따라 빌드 환경을 구성한 뒤 빌드와 린트 테스트

이 두가지의 작업이 성공적으로 끝나면 PR이 병합될 수 있는 상태로 판단해줍니다!

친절한 설명감사합니다.. 확인했습니다!

Poppool/.swiftlint.yml

Poppool/Poppool/Resource/Info.plist

zzangzzangguy · 2025-04-05T03:51:18Z

.github/workflows/ci.yml

+            echo "No changes to commit"
+          fi
+
+  build:


친절한 설명감사합니다.. 확인했습니다!

0Hooni added 18 commits March 30, 2025 18:44

remove/#93: 컨벤션 파일 삭제

f7e5439

chore/#93: 빌드 파일 수정

4795a0a

- 컨벤션 파일 제거로 인한 빌드 파일 수정

chore/#93: 프로젝트 세팅 수정

847a725

- Run SwiftLint 스크립트 추가 - SwiftLint rule 파일 추가

add/#93: swiftlint rule 파일 추가

6cec4f8

chore/#93: SwiftLint rule 파일 기본 세팅

5782305

remove#93: 불필요한 파일 삭제

1541948

docs/#93: SwiftLint 문법 수정

8e0e591

feat/#93: Swiftlint 자동 수정 깃허브 액션 추가

ab6df32

remove/#93: 사용하지 않는 테스트 타겟 제거

3fd8a70

feat/#93: CI 깃허브 액션 추가

e3cdf10

- Build 테스트 포함 - Swiftlint 테스트 포함

fix/#93: CI가 깃액션 봇의 커밋은 무시하도록 수정

4420daf

chore/#93: Debug.xcconfig로 대체하기 위한 빌드 설정 수정

2fcd2cf

- Debug.xcconfig를 Resource 폴더에 추가함 - 해당 파일은 ignore로 걸러지도록 함 - info에 Key의 이름을 등록해두어 Bundle로 찾도록 설정

chore/#93: gitignore 수정 및 정리

37a7022

- 새로운 Debug.xcconfig를 무시하도록 수정 - 기존 ignore 설정을 보기 쉽게 정리

refactor/#93: 변경 된 키 저장 방식에 맞게 키 호출 코드 수정

46b9311

- Secrets.swift의 enum을 열도록 할 예정 - 해당 파일에서 Key 호출을 위한 여러 컴퓨티드 프로퍼티 제공 - 변경된 프로퍼티에 맞게 호출 코드도 일부 수정

refactor/#93: GitHubActions 파일 추가 수정

7af96b8

- 깃헙 액션에 맞게 파일 이름 수정 - CI → ci - Swiftlint autocorrect → swiftlint-autocorrect - 추가로 autocorrect의 작동 시기를 Push → PR open으로 수정하여 깃액션 작동 빈도를 줄임

chore/#93: CI에 Debug.xcconfig 생성 스텝 추가

fb7918f

refactor/#93: Secrets.swift 제거

b00d50b

- Secrets enum을 대체할 KeyPath enum을 추가 - gitignore에서 Secrets.swift 추적 제한 제거 - 빌드 파일 수정

refactor/#93: Secrets enum → KeyPath enum 변경 대응

f958ff6

0Hooni added 🚀 feat 새로운 기능을 추가 🔧 chore 빌드 설정, 프로젝트 설정 등 “로직에 영향 없는” 변경 🗑️ remove 파일 삭제 labels Apr 2, 2025

0Hooni requested review from dongglehada and zzangzzangguy April 2, 2025 15:08

0Hooni self-assigned this Apr 2, 2025

0Hooni linked an issue Apr 2, 2025 that may be closed by this pull request

Swiftlint & CI 적용하기 #93

Closed

0Hooni marked this pull request as draft April 2, 2025 15:10

0Hooni added 2 commits April 3, 2025 00:16

chore/#93: CI 태스크 이름 수정

334a4a3

- 몇몇 이모지가 생각보다 촌스러워서 수정함

test/#93: CI 테스트를 위한 임시 swiftlint룰 적용

2253ee8

refactor/#93: CI, AutoCorrect 통합

3d31395

0Hooni and others added 5 commits April 3, 2025 00:51

fix/#93: 린트 자동수정 명령어 수정

05e9118

fix/#93: 린트 자동수정 에러 수정 및 빌드파일 경로 탐지 추가

eac393c

fix/#93: 이슈번호 검출 정규표현식 개선 및 린트 자동수정 커밋 에러 해결

ee45fe3

fix/#93: 자동수정 커밋 액션 No upstream branch 에러 수정

50f48c8

style/#93: Apply SwiftLint autocorrect

13b2b64

0Hooni marked this pull request as ready for review April 2, 2025 16:15

0Hooni added 🔄 refactor 프로덕션 코드 리팩토링, 파일 삭제, 네이밍 수정 및 폴더링 🎨 style 코드 스타일 수정, 컨벤션 적용 🐛 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 labels Apr 3, 2025

0Hooni and others added 3 commits April 3, 2025 19:42

refactor/#93: Swiftlint 에러 수정

82931e9

style/#93: Apply SwiftLint autocorrect

81f4aaf

0Hooni linked an issue Apr 3, 2025 that may be closed by this pull request

GitAttribute를 이용해 병합 충돌 줄이기 #99

Closed

0Hooni removed a link to an issue Apr 3, 2025

GitAttribute를 이용해 병합 충돌 줄이기 #99

Closed

0Hooni changed the title ~~CI & Swiftlint autocorrect 깃허브 액션 추가~~ [FEAT] CI & Swiftlint autocorrect 깃허브 액션 추가 Apr 4, 2025

style/#93: 코드 스타일에 맞게 일부 코드 수정

7177fc0

Co-Authored-By: JunYoung <ghddns34@gmail.com>

dongglehada approved these changes Apr 4, 2025

View reviewed changes

zzangzzangguy approved these changes Apr 5, 2025

View reviewed changes

0Hooni merged commit 9ebc3b6 into dev Apr 5, 2025
2 checks passed

0Hooni deleted the style/#93-apply-CI-with-swiftlint branch April 5, 2025 03:52

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[FEAT] CI & Swiftlint autocorrect 깃허브 액션 추가#98

[FEAT] CI & Swiftlint autocorrect 깃허브 액션 추가#98
0Hooni merged 30 commits intodevfrom
style/#93-apply-CI-with-swiftlint

0Hooni commented Apr 2, 2025 •

edited

Loading

Uh oh!

0Hooni commented Apr 2, 2025

Uh oh!

0Hooni commented Apr 2, 2025

Uh oh!

dongglehada left a comment

Uh oh!

dongglehada Apr 4, 2025

Uh oh!

0Hooni Apr 4, 2025

Uh oh!

zzangzzangguy Apr 5, 2025

Uh oh!

Uh oh!

Uh oh!

zzangzzangguy Apr 5, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

0Hooni commented Apr 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📌 이슈

✅ 작업 사항

🚀 테스트 방식

Swiftlint 테스트

깃허브 액션 테스트

변경된 Key 관리 테스트

👀 ETC (추후 개발해야 할 것, 참고자료 등) ->

Uh oh!

0Hooni commented Apr 2, 2025

Uh oh!

0Hooni commented Apr 2, 2025

🥹 자동 수정이랑 빌드 테스트 전부 작동 테스트 완료 🥹

자동 수정으로 줄어들은 warning 수(4000 → 100 실화?!?!?)

Uh oh!

dongglehada left a comment

Choose a reason for hiding this comment

Uh oh!

dongglehada Apr 4, 2025

Choose a reason for hiding this comment

Uh oh!

0Hooni Apr 4, 2025

Choose a reason for hiding this comment

Uh oh!

zzangzzangguy Apr 5, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

zzangzzangguy Apr 5, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

0Hooni commented Apr 2, 2025 •

edited

Loading